Skip to content
This repository has been archived by the owner on Aug 11, 2024. It is now read-only.

Oculus Quest OpenVR support #214

Closed
wants to merge 46 commits into from
Closed

Conversation

SimonDarksideJ
Copy link
Contributor

@SimonDarksideJ SimonDarksideJ commented Jun 19, 2019

XRTK - Mixed Reality Toolkit Change Request

Overview

Adding Oculus Quest OpenVR support

Target of the change:

Is this enhancement for:

  • Platform (new or updated platform)

Changes:

Brief list of the targeted features that are being changed.

  • New controller definition
  • Platform updates

StephenHodgson and others added 30 commits May 13, 2019 17:02
enabled warnings as errors
# Conflicts:
#	XRTK-Core/Packages/com.xrtk.core/Extensions/TransformExtensions.cs
# Conflicts:
#	Submodules/SDK
#	XRTK-Core/Packages/com.xrtk.core/Inspectors/Data/EditorWindowOptions.json
@SimonDarksideJ SimonDarksideJ added the In Progress PR currently still being developed label Jun 19, 2019
@SimonDarksideJ SimonDarksideJ self-assigned this Jun 19, 2019
@SimonDarksideJ SimonDarksideJ changed the title Oculus Quest support Oculus Quest OpenVR support Jun 19, 2019
@@ -0,0 +1,33 @@
// Copyright (c) Microsoft Corporation. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these copied from the mrtk?

@@ -0,0 +1,33 @@
// Copyright (c) Microsoft Corporation. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these copied from the mrtk?

@@ -150,6 +150,7 @@ protected void UpdateSingleAxisData(MixedRealityInteractionMapping interactionMa
// If our value was updated, raise it.
if (interactionMapping.Updated)
{
Debug.LogError($"Interaction updated {interactionMapping.Description} - Value [{interactionMapping.FloatData}]");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are these errors here?

Just to verify you're getting data?

@SimonDarksideJ
Copy link
Contributor Author

Pointer issue resolved in the latest SDK PR, turns out to be a mapping issue in the profile
XRTK/com.xrtk.sdk#49

@SimonDarksideJ SimonDarksideJ removed Blocked In Progress PR currently still being developed labels Jul 10, 2019
@SimonDarksideJ SimonDarksideJ marked this pull request as ready for review July 10, 2019 16:06
@SimonDarksideJ
Copy link
Contributor Author

Closing as #234 now implements this more effectively

@SimonDarksideJ SimonDarksideJ deleted the OculusSupport branch July 10, 2019 16:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants